ci(debusine): include qli workspace for DKMS build-dependencies - #56
Merged
Bjordis Collaku (bjordiscollaku) merged 1 commit intoAug 27, 2026
Merged
Conversation
|
You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool. What Enabling Code Scanning Means:
For more information about GitHub Code Scanning, check out the documentation. |
Bjordis Collaku (bjordiscollaku)
changed the title
Bjordis Collaku (bjordiscollaku)
force-pushed
the
feat/debusine-extra-build-dep-workspaces
branch
from
July 8, 2026 22:09
4f590fd to
18f6efe
Compare
Keerthi Gowda (keerthi-go)
approved these changes
Jul 15, 2026
Christopher Obbard (obbardc)
approved these changes
Aug 14, 2026
Christopher Obbard (obbardc)
left a comment
Christopher Obbard (obbardc)
left a comment
Contributor
There was a problem hiding this comment.
Bjordis Collaku (@bjordiscollaku) Changes look good to me, but I think this commit && validation should be added to #54 && this PR closed, unless you have a reason for it to be merged separately?
The Debian Debusine build path now needs kgsl-dkms as a build dependency (via PR #54 against qcom/debian/latest). The package is published in the qli Debusine workspace, so the build step must expose qli as an extra build-dependency source when invoking debusine-action/lib/build. Set EXTRA_BUILD_DEP_WORKSPACES=qli in Build in Debusine. DEBUSINE_USER, which lib/build requires for its extra_repositories compatibility workaround, is already set on this step (added independently for the build signing key), so no change is needed for it here. Signed-off-by: Bjordis Collaku <bcollaku@qti.qualcomm.com>
Bjordis Collaku (bjordiscollaku)
force-pushed
the
feat/debusine-extra-build-dep-workspaces
branch
from
August 26, 2026 23:19
18f6efe to
1145eaa
Compare
Bjordis Collaku (bjordiscollaku)
had a problem deploying
to
Production
August 26, 2026 23:41 — with 
GitHub Actions
Error
GitHub Actions
Error
Christopher Obbard (obbardc)
approved these changes
Aug 27, 2026
Bjordis Collaku (bjordiscollaku)
merged commit Aug 27, 2026
d4a87d5
into
main
19 of 22 checks passed
Bjordis Collaku (bjordiscollaku)
temporarily deployed
to
Production
August 27, 2026 17:07 — with
GitHub Actions
Inactive
GitHub Actions
Inactive
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
PR #54 introduces DKMS module integration on the packaging branch (
qcom/debian/latest) and addskgsl-dkms (>= 1.0.2)toBuild-Depends.The Debian build path on
mainruns through Debusine (build-kernel-debusine.yml), where build-dependency resolution is scoped to configured repositories/workspaces.kgsl-dkmsis published in Debusine workspaceqli, so the build submission path must explicitly include that workspace for dependency resolution.Decision
Thread Debusine workspace dependency context from the caller workflow into
debusine-action/lib/buildby setting, in theBuild in Debusinestep:EXTRA_BUILD_DEP_WORKSPACES=qliDEBUSINE_USER=${{ secrets.DEBUSINE_USER }}lib/buildalready supportsEXTRA_BUILD_DEP_WORKSPACESand requiresDEBUSINE_USERfor its currentextra_repositoriescompatibility flow.Scope
One targeted workflow change:
.github/workflows/build-kernel-debusine.ymlBuild in Debusineenv block onlyNo packaging metadata, kernel build scripts, or Ubuntu workflow behavior changed.
Why This Unblocks DKMS
Without this change, the Debusine Debian pipeline cannot discover
kgsl-dkmsfromqli, which blocks PR #54’s DKMS-enabled build path at dependency resolution time.With this change, the Debusine submission includes
qlias an additional build-dependency source, enabling the Debian pipeline to resolve DKMS prerequisites introduced by PR #54.Validation Evidence
Validation run:
build-kernel-deb.ymlfeat/debusine-extra-build-dep-workspacespkg-linux-qcom-ref=feat/dkms-build-time-module-integrationtrixieObserved in
Build in Debusine / Build (Debusine)logs:DEBUSINE_USERis presentEXTRA_BUILD_DEP_WORKSPACES: qliis presentextra_repositoriescontains//workspace=qli/trixie@debian:suiteand is expanded to explicit repository fieldsThe run later terminated because Debusine work request
14057endedstatus=abortedduring remote execution. That abort occurred after this PR’s wiring was applied and exercised.Relationship to PR #54
mainCI workflow wiring so Debusine can resolve those requirements from the correct workspace.Both are required to make the Debian Debusine DKMS build path functional end-to-end.